Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update quickstart to support Opt::parse_args #433

Closed

Conversation

gth828r
Copy link
Contributor

@gth828r gth828r commented Oct 16, 2024

The current quickstart document suggests to use pingora v0.1, but it also suggests using the Opt::parse_args interface, which was introduced in release v0.3.0. Update the quickstart document to suggest using v0.3 when setting up the cargo.toml file.

Addresses #432

The current quickstart document suggests to use pingora v0.1, but it
also suggests using the Opt::parse_args interface, which was introduced
in release v0.3.0. Update the quickstart document to suggest using v0.3
when setting up the cargo.toml file.

Addresses cloudflare#432
@gumpt gumpt requested a review from johnhurt October 18, 2024 16:04
@johnhurt johnhurt added the Accepted This change is accepted by us and merged to our internal repo label Oct 18, 2024
johnhurt pushed a commit that referenced this pull request Oct 28, 2024
The current quickstart document suggests to use pingora v0.1, but it
also suggests using the Opt::parse_args interface, which was introduced
in release v0.3.0. Update the quickstart document to suggest using v0.3
when setting up the cargo.toml file.

Addresses #432

Includes-commit: 9153a8a
Replicated-from: #433
johnhurt pushed a commit that referenced this pull request Oct 28, 2024
The current quickstart document suggests to use pingora v0.1, but it
also suggests using the Opt::parse_args interface, which was introduced
in release v0.3.0. Update the quickstart document to suggest using v0.3
when setting up the cargo.toml file.

Addresses #432

Includes-commit: 9153a8a
Replicated-from: #433
@johnhurt
Copy link
Contributor

This has been merged as part of the weekly sync. Thank you for your contributions! 🎉

@johnhurt johnhurt closed this Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted This change is accepted by us and merged to our internal repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants